Nice setup overall — the initial migration, the round-trip tests, and the CI check are all solid. A few things I think are worth addressing before merge:

init_db.py runs both create_all() and migrations. Right now init_db() calls SQLModel.metadata.create_all(engine) and then run_migrations(). Having both kind of defeats the point of Alembic — create_all builds tables straight from the models, then migrations try to manage the same tables, and the two can drift. I think this should be run_migrations() only, dropping the create_all line, so Alembic is the single source of truth for schema. (This matters for #544 too — once I add the new models, I want Alembic managing them, not create_all creating them behind Alembic's back.)

alembic check in CI + my #544 models. Good call adding the check — but heads up that it means my #544 PR will need to include the migration for the six new models, or this check will fail the build. That's the right behavior, just flagging so we're aligned that the models PR now carries its own migration.

Small one in env.py: from app.models.models import SQLModel — SQLModel is the base class from the sqlmodel package, so this reads a bit confusingly (it works only because models.py re-exposes it). Might be cleaner as from sqlmodel import SQLModel. The Template, FormSubmission import is correct and needed for autogenerate to see them.

One note on the tests: they run against in-memory SQLite, which is great for verifying up/down/round-trip mechanics. Just worth keeping in mind that since the app runs on Postgres, migrations using Postgres-specific types (JSONB, etc. — which the #544 models will use) won't be exercised by these. Not a blocker for this PR, just relevant for what comes next.

Happy to re-review once you've had a look.